-
Notifications
You must be signed in to change notification settings - Fork 150
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Apply more ruff rules #633
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #633 +/- ##
==========================================
- Coverage 71.18% 70.90% -0.28%
==========================================
Files 14 14
Lines 1093 1093
==========================================
- Hits 778 775 -3
- Misses 315 318 +3 ☔ View full report in Codecov by Sentry. |
fdff091
to
a2bf5b9
Compare
I'm not seeing any changes to |
e52494a
to
795d5df
Compare
May I ask what prompted this change? |
9c9fa29
to
531756f
Compare
I just tried to apply some rules that raised a few valid issues in other projects. |
59869f3
to
a2c0acc
Compare
PERF401 Use a list comprehension to create a transformed list
PIE810 Call `endswith` once with a `tuple`
PLC0208 Use a sequence type instead of a `set` when iterating over values
TCH003 Move standard library import into a type-checking block
FLY002 Consider f-string instead of string join
a2c0acc
to
c95b95b
Compare
No description provided.